Skip to content

fix: commands popup hidden under UI elements (#936)#938

Open
NoahHahm wants to merge 1 commit into
siteboon:mainfrom
NoahHahm:fix/command-menu-contain-paint
Open

fix: commands popup hidden under UI elements (#936)#938
NoahHahm wants to merge 1 commit into
siteboon:mainfrom
NoahHahm:fix/command-menu-contain-paint

Conversation

@NoahHahm

@NoahHahm NoahHahm commented Jun 30, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Removes paint from contain property on .chat-composer-shell in src/index.css
  • contain: paint was creating a new stacking context that clipped the commands popup, causing it to render behind other UI elements

Root Cause

contain: paint tells the browser the element's content won't overflow its bounds, which implicitly creates a new stacking context. This prevented the commands popup from appearing above sibling elements regardless of z-index.

Fix

/* before */
contain: layout style paint;

/* after */
contain: layout style;

Test plan

  • Type / in chat input → commands popup appears above all UI elements
  • Click the commands button → popup renders correctly
  • Verify on mobile and web interface
  • Confirm regression from v1.34.0 → v1.35.0 is resolved

Closes #936

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Bug Fixes
    • Improved chat composer rendering behavior by adjusting how the interface is isolated by the browser, which may help reduce visual glitches or repaint issues.

@NoahHahm @claude
fix: remove contain paint from chat-composer-shell to fix commands po…
6fb3ec0 
…pup z-index

Removing `paint` from the CSS `contain` property fixes the commands popup
being hidden under other UI elements. `contain: paint` creates a new
stacking context that clips overflow, preventing the popup from rendering
above sibling elements.

Fixes siteboon#936

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: fcf1e793-4642-4a4e-9b02-b183e7770cef

📥 Commits

Reviewing files that changed from the base of the PR and between 2ebe64f and 6fb3ec0.

📒 Files selected for processing (1)
  • src/index.css
📝 Walkthrough

Walkthrough

In src/index.css, the contain property on .chat-composer-shell is changed from layout style paint to layout style, removing the paint containment that was causing the commands popup to be clipped by the browser's paint isolation boundary.

Changes

Commands Popup Visibility Fix

Layer / File(s) Summary
Remove paint containment
src/index.css		 contain on .chat-composer-shell drops the paint keyword, allowing the commands popup to render outside the element's paint boundary.

A bunny hopped through the CSS one day,
Found paint containment blocking the way,
The popup was hidden, clipped out of sight,
One keyword removed — now it renders right!
🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the popup layering fix and matches the main change.
Linked Issues check ✅ Passed The CSS change directly addresses #936 by removing paint containment that hid the commands popup.
Out of Scope Changes check ✅ Passed The PR contains only the targeted CSS containment change, with no unrelated additions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Commands popup is hidden under other UI elements

1 participant

@NoahHahm